Skip to content

fix(controller): treat panicking controller extension as runtime failure#3402

Draft
ThomsonTan wants to merge 2 commits into
open-telemetry:mainfrom
ThomsonTan:fix_controller_panics
Draft

fix(controller): treat panicking controller extension as runtime failure#3402
ThomsonTan wants to merge 2 commits into
open-telemetry:mainfrom
ThomsonTan:fix_controller_panics

Conversation

@ThomsonTan

@ThomsonTan ThomsonTan commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Change Summary

What issue does this PR close?

How are these changes tested?

Are there any user-facing changes?

Changelog

  • Added a .chloggen/*.yaml entry
  • This PR is a chore (indicated in title)
  • This is a documentation-only PR.

@github-actions github-actions Bot added lang:rust Pull requests that update Rust code area:engine Internal engine features labels Jul 1, 2026
@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.03922% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 86.25%. Comparing base (3a705c4) to head (e5eb044).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #3402      +/-   ##
==========================================
+ Coverage   86.23%   86.25%   +0.01%     
==========================================
  Files         739      741       +2     
  Lines      292341   292872     +531     
==========================================
+ Hits       252107   252619     +512     
- Misses      39710    39729      +19     
  Partials      524      524
Components Coverage Δ
otap-dataflow 87.31% <98.03%> (+0.01%) ⬆️
query_engine 89.58% <ø> (ø)
otel-arrow-go 52.45% <ø> (ø)
quiver 92.19% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

area:engine Internal engine features lang:rust Pull requests that update Rust code

Projects

OTel-Arrow
Status: No status

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Controller extension panic bypasses fail-fast shutdown and wedges the engine in run_forever

1 participant

@ThomsonTan